-
Notifications
You must be signed in to change notification settings - Fork 446
update pyproject.toml to support Python 3.14 #772
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #772 +/- ##
==========================================
- Coverage 88.86% 88.84% -0.02%
==========================================
Files 7 7
Lines 548 556 +8
==========================================
+ Hits 487 494 +7
- Misses 61 62 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Ofc this breaks compatibility with all previous python versions. Suggestions welcome as I don't have a lot of time and are quite new to building packages for pip |
setup.py
Outdated
| connector = os.path.join( | ||
| os.environ["ProgramFiles"], "MariaDB", "MariaDB Connector C" | ||
| os.environ["ProgramFiles"], "MariaDB", "MariaDB Connector C", | ||
| os.environ["ProgramFiles"], "MariaDB", "MariaDB Connector C 64-bit", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have Windows box now. Is this current default location of official installer?
Updated the required Python version from 3.8 to 3.10 and removed support for 3.8 and 3.9.
Remove redundant path for MariaDB Connector C.
Suggested changes to fix deprecation and changes to format of project.license in python 3.14: C:\Users\XXX\AppData\Local\Temp\pip-build-env-cc_fwjqb\overlay\Lib\site-packages\setuptools\config_apply_pyprojecttoml.py:82: SetuptoolsDeprecationWarning:
project.licenseas a TOML table is deprecated!!
And also to fix build errors when using the 64 bit MariaDB Connector C, but not sure if this is the correct way.